From: Florian Müllner <fmuellner@gnome.org>
Date: Fri, 3 Feb 2023 14:52:46 +0000 (+0100)
Subject: st/icon-theme: Remove unnecessary CLAMP()
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~371^2^2~100
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=02fc765baa9ffb49f3e041f72bd2eb25d766a950;p=gnome-shell.git

st/icon-theme: Remove unnecessary CLAMP()

The ClutterColor struct members are 8-bit unsigned integers, so
they are always in the 0 to 255 range.

Spotted by coverity.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2679>
---

diff --git a/src/st/st-icon-theme.c b/src/st/st-icon-theme.c
index a7cbd0da8..dc3f8cedc 100644
--- a/src/st/st-icon-theme.c
+++ b/src/st/st-icon-theme.c
@@ -3659,9 +3659,9 @@ static char *
 color_to_string_noalpha (const ClutterColor *color)
 {
   return g_strdup_printf ("rgb(%d,%d,%d)",
-                          CLAMP (color->red, 0, 255),
-                          CLAMP (color->green, 0, 255),
-                          CLAMP (color->blue, 0, 255));
+                          color->red,
+                          color->green,
+                          color->blue);
 }
 
 static void